Skip to content

Build arm64 image on GitHub Actions #39

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 8 commits into from
Mar 4, 2022

Conversation

tysg
Copy link

@tysg tysg commented Mar 4, 2022

A part of #34

GitHub Actions' yaml is super hard to debug locally, so expect this PR to contain lots of frustrated commits :)

@tysg
Copy link
Author

tysg commented Mar 4, 2022

Oops the latest workflow is running for over an hour. @zamazan4ik could you cancel it, or does it matter? I'll address this in the next commit.

@zamazan4ik
Copy link

Don`t worry - it will ba cancelled automatically

@tysg
Copy link
Author

tysg commented Mar 4, 2022

Looking at this line https://github.com/rust-serverless/lambda-rust/runs/5419998450?check_suite_focus=true#step:4:539,

  1.58.1-aarch64-unknown-linux-gnu installed - rustc 1.58.1 (db9d1b20b 2022-01-20)

I'm reasonably confident that the image is building on QEMU. Currently we only run the tests against the x64 image, since running the test on QEMU is too slow.

I'll run the test with the arm64 image on my local machine once it's published. @zamazan4ik sorry for the GitHub workflow spam, please feel free to make any edit, too! (Also please squash if you're going to merge it, I'll be too embarrassed to see the commit messages on the master branch)

@tysg tysg changed the title WIP: Attempt to build arm64 on GitHub Actions Build arm64 image on GitHub Actions Mar 4, 2022
@tysg tysg marked this pull request as ready for review March 4, 2022 09:40
@zamazan4ik zamazan4ik merged commit 0f6d8da into rust-serverless:master Mar 4, 2022
@zamazan4ik
Copy link

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants